Skip to content

Fix mx unittest launch for configure tests - #13193

Closed
oleksii-tumanov wants to merge 1 commit into
oracle:masterfrom
oleksii-tumanov:fix-configure-unittest-launch
Closed

Fix mx unittest launch for configure tests#13193
oleksii-tumanov wants to merge 1 commit into
oracle:masterfrom
oleksii-tumanov:fix-configure-unittest-launch

Conversation

@oleksii-tumanov

@oleksii-tumanov oleksii-tumanov commented Mar 24, 2026

Copy link
Copy Markdown
Member

Summary

  • Make the existing SVM unit-test launch configuration reusable.
  • Apply it to SVM_TESTS so configure tests receive the required module exports and package opens.
  • On current master, mx unittest ResourceConfigurationTest fails during JUnit discovery with an IllegalAccessError.

Related Issues

Testing

  • mx build --dependencies=SVM_TESTS --build-logs=silent
  • mx unittest ResourceConfigurationTest (OK (3 tests))
  • mx unittest com.oracle.svm.configure.test (OK (26 tests))
  • mx checkstyle

Documentation

  • No documentation updates are needed because this only changes the local unit-test launch configuration.

Contributor Checklist

  • I have read the contribution guide.
  • I have the right to contribute the submitted material under the project terms.
  • I have updated tests and documentation where appropriate.
  • If I used a coding assistant, I remain responsible for the entire contribution and have reviewed it accordingly.

@oracle-contributor-agreement oracle-contributor-agreement Bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Mar 24, 2026
@oleksii-tumanov

Copy link
Copy Markdown
Member Author

issue found during #13195 tests

@oleksii-tumanov
oleksii-tumanov marked this pull request as ready for review March 24, 2026 04:25
@oleksii-tumanov
oleksii-tumanov force-pushed the fix-configure-unittest-launch branch from 42071a7 to 8f346d2 Compare March 24, 2026 20:39
@oleksii-tumanov
oleksii-tumanov force-pushed the fix-configure-unittest-launch branch from 8f346d2 to ba9ea7d Compare June 22, 2026 10:35
@oleksii-tumanov
oleksii-tumanov force-pushed the fix-configure-unittest-launch branch from ba9ea7d to 41f411a Compare July 31, 2026 00:30
@oleksii-tumanov

Copy link
Copy Markdown
Member Author

Closing after rechecking the intended test workflow. SVM_TESTS is designed to run through native-unittest, while JVM configure-test paths provide their required exports explicitly. This PR would enable an unsupported invocation rather than fix missing test coverage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant